[IMP] accounting: update peppol information on e-invoicing page #13922
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
afma-odoo
force-pushed
the
17.0-accounting-peppol-afma
branch
from
f85fe0e to
dd8d039
Compare
auva-odoo
requested changes
Jul 9, 2025
There was a problem hiding this comment.
Hello @afma-odoo thank you for tackling this task, it was much, much needed!
In addition to my review comments:
- The Demo/Live/Test radio buttons were removed so I don't think "demo mode", "live mode", etc. need to use guilabel and be capitalized? I mentioned this a few times in my review comments but you might want to check the whole Peppol section just in case?
- Unless it is a UI element (and using a guilabel), I don't think the word "endpoint" needs to have a capital e. Could you please chekc the whole doc?
Thank you again for your work! (and your patience while I was figuring out the testing 🙈 )
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/accounting/customer_invoices/electronic_invoicing.rst
Outdated
Show resolved
Hide resolved
afma-odoo
force-pushed
the
17.0-accounting-peppol-afma
branch
from
dd8d039 to
0bd5c47
Compare
|
@auva-odoo Thank you for your comments/suggestions. |
task-4737738 task-4794170 task-4360579
auva-odoo
force-pushed
the
17.0-accounting-peppol-afma
branch
from
0bd5c47 to
c5da52e
Compare
auva-odoo
approved these changes
Jul 14, 2025
There was a problem hiding this comment.
Thank you @afma-odoo ! FYI I added a section for the test mode as I thought it was too long for a tip.
@robodoo r+
robodoo
pushed a commit
that referenced
this pull request
Jul 14, 2025
task-4737738 task-4794170 task-4360579 closes #13922 Signed-off-by: Audrey Vandromme (auva) <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
task-4737738
task-4794170
task-4360579